Skip to content

Migrate sonic-mgmt-framework container to Debian Trixie#26548

Merged
yxieca merged 13 commits into
sonic-net:masterfrom
amrutasali:trixie_migration_for_sonic_mgmt_framework
May 1, 2026
Merged

Migrate sonic-mgmt-framework container to Debian Trixie#26548
yxieca merged 13 commits into
sonic-net:masterfrom
amrutasali:trixie_migration_for_sonic_mgmt_framework

Conversation

@amrutasali
Copy link
Copy Markdown
Contributor

@amrutasali amrutasali commented Apr 3, 2026

Why I did it - Required for sonic-net/SONiC#2169

How I did it - Updated the Docker build files to build for Debian Trixie. The dependency packages are updated accordingly.

How to verify it - The sonic-mgmt-framework container builds successfully under BLDENV=trixie. Installed it on a recent SONiC image and verified that the docker os distribution shows Trixie and basic curl test on REST server hosted within the docker works fine.

Note: The sonic-mgmt-framework build for trixie has a dependency on below PRs and then a submodule update for the 2 repos :
sonic-net/sonic-mgmt-common#211
sonic-net/sonic-mgmt-framework#158

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Signed-off-by: Amruta Sali <amruta_sali@dell.com>
@amrutasali amrutasali force-pushed the trixie_migration_for_sonic_mgmt_framework branch from 75b8c91 to 8cd8fab Compare April 3, 2026 21:58
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@amrutasali
Copy link
Copy Markdown
Contributor Author

@saiarcot895 Please add this to tracker - sonic-net/SONiC#2169 .
Note : sonic-gnmi(bump Golang to 1.24.4) repo PR will be opened next week

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates the docker-sonic-mgmt-framework container to build on the Debian Trixie base (via docker-config-engine-trixie), aligning the container’s build/runtime environment with the newer BLDENV=trixie build flow.

Changes:

  • Switch docker-sonic-mgmt-framework build dependencies/load base from docker-config-engine-bookworm to docker-config-engine-trixie.
  • Update the container Dockerfile base image to docker-config-engine-trixie-* and adjust installed Debian packages accordingly.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
rules/docker-sonic-mgmt-framework.mk Moves the mgmt-framework docker image’s base/load/debug dependency wiring from bookworm config-engine to trixie config-engine, and reclassifies it under “trixie” docker lists.
dockers/docker-sonic-mgmt-framework/Dockerfile.j2 Switches the Docker build ARG base image to docker-config-engine-trixie-* and updates apt dependencies for Trixie.

Comment thread rules/docker-sonic-mgmt-framework.mk
@saiarcot895
Copy link
Copy Markdown
Contributor

@saiarcot895 Please add this to tracker - sonic-net/SONiC#2169 . Note : sonic-gnmi(bump Golang to 1.24.4) repo PR will be opened next week

@amrutasali Please note #26499.

@amrutasali
Copy link
Copy Markdown
Contributor Author

amrutasali commented Apr 3, 2026

@saiarcot895 Please add this to tracker - sonic-net/SONiC#2169 . Note : sonic-gnmi(bump Golang to 1.24.4) repo PR will be opened next week

@amrutasali Please note #26499.

@saiarcot895 yes I'm aware and got it confirmed - #25959 (comment) .So will just bump Golang version for soni-gnmi repository.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

…mpiled against cjson headers inside the container

Signed-off-by: Amruta Sali <amruta_sali@dell.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Apr 14, 2026

/azpw ms_conflict

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Apr 14, 2026

/azpw ms_conflict

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Apr 15, 2026

/azpw ms_conflict

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

…o 1.24.4 (trixie-only) and are only used by trixie-specific docker images. Add BLDENV=trixie conditionals following the libpcre3 pattern to avoid bookworm build failures.

Signed-off-by: Amruta Sali <amruta_sali@dell.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@amrutasali
Copy link
Copy Markdown
Contributor Author

@saiarcot895 this is ready to be merged

Comment thread rules/sonic-mgmt-common.mk Outdated
@hdwhdw
Copy link
Copy Markdown
Contributor

hdwhdw commented Apr 27, 2026

@yxieca ping for merge. This blocks sonic-mgmt-common submodule update, which in turn blocks sonic-gnmi submodule (already 3 weeks old). @saiarcot895 is this PR good for merge?

…equire Go 1.24.4 (trixie-only) and are only used by trixie-specific docker images. Add BLDENV=trixie conditionals following the libpcre3 pattern to avoid bookworm build failures."

This reverts commit 9585d23.

Signed-off-by: Amruta Sali <amruta_sali@dell.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@yxieca yxieca merged commit 9850e53 into sonic-net:master May 1, 2026
27 checks passed
mhchann pushed a commit to mhchann/sonic-buildimage that referenced this pull request May 7, 2026
)

What: Migrated docker-sonic-mgmt-framework container from Debian Bookworm to Trixie.
Why: Required for the Trixie migration effort (sonic-net/SONiC#2169).
How: Updated Docker build files for Trixie. Updated dependency packages accordingly. Dependencies: sonic-mgmt-common/pull/211 and sonic-mgmt-framework/pull/158 with submodule updates.
Testing: Container builds successfully under BLDENV=trixie. Verified docker OS shows Trixie and basic curl test on REST server works. All CI builds and KVM tests passed.

Signed-off-by: Amruta Sali <amruta_sali@dell.com>
Signed-off-by: mhchann <mhchann082@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants